Skip to content

Add codex plugin#1482

Merged
ChiragAgg5k merged 17 commits intomasterfrom
codex-plugin
May 8, 2026
Merged

Add codex plugin#1482
ChiragAgg5k merged 17 commits intomasterfrom
codex-plugin

Conversation

@adityaoberai
Copy link
Copy Markdown
Contributor

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR adds a Codex CLI plugin generator to the Appwrite SDK generator, following the same pattern as the existing CursorPlugin and ClaudePlugin. It introduces templates for the plugin manifest, marketplace manifest, MCP config, config.toml snippet, and README, along with the CodexPlugin PHP class that wires everything together.

  • CodexPlugin.php manually builds the file list using agent-skills/ templates directly (avoiding any skillName frontmatter mismatch), places skills under plugins/appwrite/skills/appwrite-<lang>/SKILL.md, and references plugin/LICENSE.twig correctly.
  • plugin.json.twig uses {{ spec.licenseName }} and {{ sdk.gitUserName }}/{{ sdk.gitRepoName }} template variables; relative paths ../skills/ and ../.mcp.json resolve correctly from .codex-plugin/ to the plugin root.
  • marketplace.json.twig is placed at .agents/plugins/marketplace.json and uses "../../plugins/appwrite" to traverse up to the repo root and then into the plugin directory.

Confidence Score: 5/5

Safe to merge — the new plugin generator is self-contained and all previously flagged path, license, and template issues have been resolved.

All changed files are additive (new templates and a new PHP class). The implementation correctly uses agent-skills/ templates directly to avoid frontmatter name mismatches, fixes the LICENSE template reference to plugin/LICENSE.twig, and applies proper relative paths throughout.

No files require special attention.

Important Files Changed

Filename Overview
src/SDK/Language/CodexPlugin.php New plugin class extending AgentSkills; manually builds file list using agent-skills templates directly (avoiding skillName frontmatter mismatch), correct LICENSE template reference, and correct subdirectory placement under plugins/appwrite/.
templates/codex-plugin/plugin.json.twig Plugin manifest with correct template variables for license and repository; relative paths "../skills/" and "../.mcp.json" correctly resolve from .codex-plugin/ to plugins/appwrite/skills/ and plugins/appwrite/.mcp.json respectively.
templates/codex-plugin/marketplace.json.twig Marketplace manifest generated at .agents/plugins/marketplace.json; path "../../plugins/appwrite" correctly traverses two levels up to the repo root then into plugins/appwrite/ where the plugin lives.
templates/codex-plugin/README.md.twig README with installation instructions that correctly reference plugins/appwrite/ paths matching the generated file structure; structure diagram is accurate.
templates/codex-plugin/.mcp.json.twig MCP config for plugin-based installs using native HTTP transport; consistent with config.toml which uses npx mcp-remote for manual installs.
templates/codex-plugin/config.toml.twig Config snippet for manual installs using npx mcp-remote; correctly paired with the plugin's .mcp.json which uses native HTTP for plugin-managed installs.
example.php Adds CodexPlugin generation with BSD-3-Clause license, correctly added to speclessSDKs list so no API spec is required at generation time.
AGENTS.md Documentation-only update adding CodexPlugin to the supported SDKs list and the SDK target table.

Reviews (15): Last reviewed commit: "Reuse shared deploy templates for Codex ..." | Re-trigger Greptile

Comment thread templates/codex-plugin/skills/deploy-site.md.twig Outdated
Comment thread templates/codex-plugin/plugin.json.twig Outdated
Comment thread templates/codex-plugin/skills/cli.md.twig Outdated
adityaoberai and others added 4 commits April 28, 2026 21:46
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Comment thread templates/codex-plugin/skills/cli.md.twig Outdated
Comment thread templates/codex-plugin/plugin.json.twig Outdated
Comment thread templates/codex-plugin/marketplace.json.twig
Comment thread templates/codex-plugin/plugin.json.twig Outdated
Comment thread templates/codex-plugin/README.md.twig
adityaoberai and others added 3 commits April 30, 2026 02:20
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Comment thread templates/codex-plugin/marketplace.json.twig
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Comment thread templates/cursor-plugin/LICENSE.twig Outdated
@@ -1,21 +1,12 @@
MIT License
Copyright (c) 2026 Appwrite (https://appwrite.io) and individual contributors.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why did we change license

Comment thread src/SDK/Language/CodexPlugin.php
@ChiragAgg5k ChiragAgg5k merged commit e363fff into master May 8, 2026
57 checks passed
@ChiragAgg5k ChiragAgg5k deleted the codex-plugin branch May 8, 2026 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants